Skip to content

Fix non-native macOS cursor#14101

Merged
larsoner merged 9 commits into
mne-tools:mainfrom
larsoner:repro
Jul 25, 2026
Merged

Fix non-native macOS cursor#14101
larsoner merged 9 commits into
mne-tools:mainfrom
larsoner:repro

Conversation

@larsoner

@larsoner larsoner commented Jul 24, 2026

Copy link
Copy Markdown
Member

I think I isolated the pytest-xdist problem on macOS. This first commit should come back red on a hang as before. Then I'll switch the installed pytest-xdist to mine, and hope it fixes things 🤞

DRAFT, DO NOT MERGE

If I make an upstream PR and it's merged, I'll update this PR to start using the fixed version until they cut a release.

@larsoner

larsoner commented Jul 24, 2026

Copy link
Copy Markdown
Member Author

Okay stuck on mne/viz/_3d.py::mne.viz._3d.plot_volume_source_estimates at 21m mark. I'll kill it anytime I notice after 25m and switch to my branch 🤞

@larsoner

larsoner commented Jul 24, 2026

Copy link
Copy Markdown
Member Author

Okay, it died rather than hung, which is a good thing for pytest-xdist (no longer hanging0 but maybe not for us. I'll have to investigate further what's causing the coreg test to die!

@larsoner larsoner mentioned this pull request Jul 25, 2026
Open
2 tasks
@larsoner
larsoner marked this pull request as ready for review July 25, 2026 15:01
@larsoner
larsoner requested a review from drammock as a code owner July 25, 2026 15:01
@larsoner larsoner changed the title Test pytest-xdist fix Fix non-native macOS cursor Jul 25, 2026
@larsoner

larsoner commented Jul 25, 2026

Copy link
Copy Markdown
Member Author

Okay this now uses a native cursor on macOS for "don't interact, GUI busy". It turns out the non-native "busy" string caused a Qt<->Cocoa bug to get triggered and segfault. And this then hit the pytest-xdist bug. So by using a native cursor (good fix anyway), we can then use a single-pass macOS arm64 test structure. This very complicated situation turns out to have a very trivial fix (/ improvement) at our end so I'll self-mark for merge-when-green!

@larsoner
larsoner enabled auto-merge (squash) July 25, 2026 15:04
@larsoner
larsoner merged commit b6efc8a into mne-tools:main Jul 25, 2026
29 checks passed
@larsoner
larsoner deleted the repro branch July 25, 2026 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@drammock drammock Awaiting requested review from drammock drammock is a code owner

Assignees

No one assigned

Labels

no-changelog-entry-needed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@larsoner